Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Handle styling of varying amounts of DOJO exercise previews #2341

Closed
wants to merge 1 commit into from
Closed

Handle styling of varying amounts of DOJO exercise previews #2341

wants to merge 1 commit into from

Conversation

bryanjenningz
Copy link
Collaborator

@bryanjenningz bryanjenningz commented Sep 23, 2022

This pull request fixes some of the styling for the DOJO exercise previews so that they look good with any amount.

With 8 exercise previews:
exercise-previews

With 4 exercise previews:
exercise-previews-2

How to test:

  • Go to /exercises/js0
  • Check that the exercise previews look good

This is a part of #2251

@vercel
Copy link

vercel bot commented Sep 23, 2022

@bryanjenningz is attempting to deploy a commit to the c0d3-prod Team on Vercel.

A member of the Team first needs to authorize it.

@vercel
Copy link

vercel bot commented Sep 23, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
c0d3-app ✅ Ready (Inspect) Visit Preview Sep 23, 2022 at 9:46PM (UTC)

/>
))}
</div>
{chunk(mockExercisePreviews, 3).map((exercisePreviewChunk, i) => (
Copy link
Collaborator Author

@bryanjenningz bryanjenningz Sep 23, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This works well for 1000px width screens and above. We'll need to handle smaller screens in the future when the designs are ready.

@bryanjenningz
Copy link
Collaborator Author

Closing this pull request because I created a new pull request that makes the DOJO exercise list responsive to all screen widths: #2342

@bryanjenningz bryanjenningz deleted the handle-variable-amount-of-exercise-previews branch September 24, 2022 17:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant